Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Guided tour of exploration page #737

Merged
merged 4 commits into from
Nov 10, 2023
Merged

Conversation

danielfdsilva
Copy link
Collaborator

I've been exploring user react tour as a way to create a guided tour of the exploration page. I added a couple of steps just to show case how we could use it. There are a lot more to add, also after the user enter the "analysis mode.

@j08lue @ricardoduplos what do you think of this?

Copy link

netlify bot commented Nov 7, 2023

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit cd20ed2
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui/deploys/654e3f62ef5ec900088a6363
😎 Deploy Preview https://deploy-preview-737--veda-ui.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@j08lue j08lue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I understood @ricardoduplos, it is a deliberate design choice to keep the controls very simple, without a lot of annotations and explaining labels. 👌

I think in this case it is necessary to introduce essential features like the ones you highlighted with the guided tour here, @danielfdsilva.

The tour looks great, @danielfdsilva, and I agree with your choice of features to highlight. I am excited to see how our users respond to these.

I made some copy-editing suggestions on the texts - please review them and accept if you agree with my changes.

app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
@danielfdsilva
Copy link
Collaborator Author

@j08lue Added another tour when the user draws an AOI. Please give the messages some love :)

@danielfdsilva danielfdsilva marked this pull request as ready for review November 9, 2023 16:54
@danielfdsilva danielfdsilva requested a review from j08lue November 10, 2023 13:54
Copy link
Contributor

@j08lue j08lue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More copy editing for tour messages - I made some of them shorter and used the same type of phrase, presenting the opportunities ("you can ..."). 🤷

app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
app/scripts/components/exploration/tour-manager.tsx Outdated Show resolved Hide resolved
@danielfdsilva danielfdsilva merged commit a7d15b4 into feature/exploration Nov 10, 2023
5 checks passed
@danielfdsilva danielfdsilva deleted the feature/user-aids branch November 10, 2023 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants